Skip to content

ci: enforce tool metadata sync - #43

Open
alloutflo wants to merge 2 commits into
Railly:mainfrom
alloutflo:codex/issue-37-tool-sync-check
Open

ci: enforce tool metadata sync#43
alloutflo wants to merge 2 commits into
Railly:mainfrom
alloutflo:codex/issue-37-tool-sync-check

Conversation

@alloutflo

@alloutflo alloutflo commented Aug 9, 2026

Copy link
Copy Markdown

Summary

  • Compare the runtime TOOL_CONFIGS roster with display-facing TOOL_NAMES inside the check already run by CI.
  • Exclude the documented global-agents shared directory from the coding-tool count.
  • Report count drift plus missing or stale names while preserving the existing README count validation.

Closes #37

Validation

  • bun install --frozen-lockfile
  • bun run scripts/check-tool-count.ts
  • bun run lint
  • bun run build
  • Injected an extra runtime config through a Bun module mock and verified the script exits 1 with the missing TOOL_NAMES entry named in the diagnostic.

Draft gates

  • GitHub Actions CI reports action_required; a maintainer must approve the first-time fork workflow run before it can execute.
  • The Vercel status reports failure at a team authorization URL, so a maintainer must authorize or waive that external deployment check.

After both remote gates are resolved and machine checks are green, the PR can be marked ready.

@vercel

vercel Bot commented Aug 9, 2026

Copy link
Copy Markdown

@alloutflo is attempting to deploy a commit to the Crafter Station Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Copy Markdown
Author

@Railly Maintainer handoff: contributor-side work is complete at 32eeebc4. The PR is mergeable, up to date with main, and has no review comments or open threads. The tool-roster check, lint, and production build pass locally, including a negative test that proves roster drift fails with the missing name in the diagnostic.

Two maintainer-side gates remain: approve the CI workflow, which is action_required with 0 jobs, and authorize or waive the Vercel deployment check. I’ll keep the PR in draft until the required remote checks are green.

@Railly
Railly marked this pull request as ready for review August 10, 2026 17:36
@vercel

vercel Bot commented Aug 10, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
agenfiles-web Ready Ready Preview Aug 10, 2026 5:36pm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CI check: keep TOOL_NAMES (tools-meta.ts) in sync with tool-configs.ts

1 participant